Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SQL] Fix logging warn -> debug #1800

Closed
wants to merge 2 commits into from
Closed

Conversation

marmbrus
Copy link
Contributor

@marmbrus marmbrus commented Aug 6, 2014

No description provided.

Minor refactoring to allow resolution either using a nodes input or output.

Author: Michael Armbrust <michael@databricks.com>

Closes apache#1795 from marmbrus/ordering and squashes the following commits:

237f580 [Michael Armbrust] style
74d833b [Michael Armbrust] newline
705d963 [Michael Armbrust] Add a rule for resolving ORDER BY expressions that reference attributes not present in the SELECT clause.
82cabda [Michael Armbrust] Generalize attribute resolution.
@SparkQA
Copy link

SparkQA commented Aug 6, 2014

QA tests have started for PR 1800. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17988/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 6, 2014

QA results for PR 1800:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/17988/consoleFull

@marmbrus
Copy link
Contributor Author

marmbrus commented Aug 6, 2014

Merged.

@asfgit asfgit closed this in 5a826c0 Aug 6, 2014
asfgit pushed a commit that referenced this pull request Aug 6, 2014
Author: Michael Armbrust <michael@databricks.com>

Closes #1800 from marmbrus/warning and squashes the following commits:

8ea9cf1 [Michael Armbrust] [SQL] Fix logging warn -> debug.

(cherry picked from commit 5a826c0)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@marmbrus marmbrus deleted the warning branch August 27, 2014 20:57
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Author: Michael Armbrust <michael@databricks.com>

Closes apache#1800 from marmbrus/warning and squashes the following commits:

8ea9cf1 [Michael Armbrust] [SQL] Fix logging warn -> debug.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants